Skip to content

[fix](compile) ignore gen_cpp compile warnings #55581

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Closed
shuke987 wants to merge 3 commits into from
Closed

[fix](compile) ignore gen_cpp compile warnings #55581

shuke987 wants to merge 3 commits into from

Conversation

@shuke987
Copy link
Collaborator

@shuke987 shuke987 commented Sep 2, 2025

What problem does this PR solve?

Issue Number: close #xxx

Related PR: #xxx

Problem Summary:

Release note

None

Check List (For Author)

  • Test

    • Regression test
    • Unit Test
    • Manual test (add detailed scripts or steps below)
    • No need to test or manual test. Explain why:
      • This is a refactor/code format and no logic has been changed.
      • Previous test can cover this change.
      • No code files have been changed.
      • Other reason

  • Behavior changed:

    • No.
    • Yes.

  • Does this need documentation?

    • No.
    • Yes.

Check List (For Reviewer who merge this PR)

  • Confirm the release note
  • Confirm test cases
  • Confirm document
  • Add branch pick label

@Thearas
Copy link
Contributor

Thearas commented Sep 2, 2025

Thank you for your contribution to Apache Doris.
Don't know what should be done next? See How to process your PR.

Please clearly describe your PR:

  1. What problem was fixed (it's best to include specific error reporting information). How it was fixed.
  2. Which behaviors were modified. What was the previous behavior, what is it now, why was it modified, and what possible impacts might there be.
  3. What features were added. Why was this function added?
  4. Which code was refactored and why was this part of the code refactored?
  5. Which functions were optimized and what is the difference before and after the optimization?

@shuke987
Copy link
Collaborator Author

shuke987 commented Sep 2, 2025

run compile

@shuke987
Copy link
Collaborator Author

shuke987 commented Sep 4, 2025

run buildall

@doris-robot
Copy link

doris-robot commented Sep 4, 2025

Cloud UT Coverage Report

Increment line coverage 🎉

Increment coverage report
Complete coverage report

Category Coverage
Function Coverage 84.73% (1481/1748)
Line Coverage 68.04% (26549/39022)
Region Coverage 68.91% (13140/19069)
Branch Coverage 58.90% (7039/11950)

@doris-robot
Copy link

doris-robot commented Sep 4, 2025

TPC-H: Total hot run time: 34407 ms 
machine: 'aliyun_ecs.c7a.8xlarge_32C64G'
scripts: https://github.com/apache/doris/tree/master/tools/tpch-tools
Tpch sf100 test result on commit 9a09e9bb45b9dd4df851767414643c079614e0f7, data reload: false

------ Round 1 ----------------------------------
q1	17627	5363	5115	5115
q2	2048	341	222	222
q3	10227	1297	738	738
q4	10231	1042	562	562
q5	7539	2431	2383	2383
q6	191	170	142	142
q7	936	785	624	624
q8	9352	1354	1180	1180
q9	7045	5150	5220	5150
q10	6914	2405	1964	1964
q11	548	305	279	279
q12	357	367	230	230
q13	17785	3715	3065	3065
q14	239	252	230	230
q15	572	520	509	509
q16	438	428	376	376
q17	603	882	368	368
q18	7407	7128	7048	7048
q19	1107	968	579	579
q20	366	352	247	247
q21	3973	2640	2398	2398
q22	1079	1051	998	998
Total cold run time: 106584 ms
Total hot run time: 34407 ms

----- Round 2, with runtime_filter_mode=off -----
q1	5225	5158	5123	5123
q2	252	344	230	230
q3	2173	2712	2337	2337
q4	1365	1797	1391	1391
q5	4212	4399	4630	4399
q6	229	177	139	139
q7	2102	1979	1837	1837
q8	2694	2679	2649	2649
q9	7547	7253	7385	7253
q10	3098	3519	2868	2868
q11	591	515	483	483
q12	686	833	687	687
q13	3605	4143	3305	3305
q14	312	319	299	299
q15	529	481	467	467
q16	486	508	480	480
q17	1270	1628	1392	1392
q18	7962	7697	7454	7454
q19	872	905	1024	905
q20	2013	2122	1904	1904
q21	5257	4531	4537	4531
q22	1155	1053	1007	1007
Total cold run time: 53635 ms
Total hot run time: 51140 ms

@doris-robot
Copy link

doris-robot commented Sep 4, 2025

TPC-DS: Total hot run time: 187789 ms 
machine: 'aliyun_ecs.c7a.8xlarge_32C64G'
scripts: https://github.com/apache/doris/tree/master/tools/tpcds-tools
TPC-DS sf100 test result on commit 9a09e9bb45b9dd4df851767414643c079614e0f7, data reload: false

query1	1069	450	434	434
query2	6572	1811	1725	1725
query3	6759	229	227	227
query4	26202	23420	23103	23103
query5	4874	693	558	558
query6	355	266	238	238
query7	4674	533	320	320
query8	312	269	264	264
query9	8634	2920	2902	2902
query10	512	352	301	301
query11	16094	15176	14873	14873
query12	186	122	117	117
query13	1671	567	436	436
query14	11160	6058	5965	5965
query15	268	198	178	178
query16	7768	687	487	487
query17	1629	813	667	667
query18	2062	454	356	356
query19	284	231	189	189
query20	140	136	124	124
query21	217	139	121	121
query22	4127	4163	4152	4152
query23	33978	33268	32965	32965
query24	7745	2434	2444	2434
query25	576	540	457	457
query26	1230	286	176	176
query27	2659	533	363	363
query28	4326	2304	2277	2277
query29	755	617	510	510
query30	298	228	204	204
query31	920	856	767	767
query32	94	84	80	80
query33	588	421	372	372
query34	806	863	532	532
query35	837	831	762	762
query36	975	1028	934	934
query37	138	125	96	96
query38	4122	4055	4120	4055
query39	1502	1445	1440	1440
query40	228	143	135	135
query41	69	65	63	63
query42	131	118	116	116
query43	506	538	480	480
query44	1367	896	870	870
query45	189	181	176	176
query46	890	1026	654	654
query47	1760	1807	1729	1729
query48	389	453	335	335
query49	785	561	414	414
query50	678	722	408	408
query51	4115	4179	4142	4142
query52	126	121	113	113
query53	263	283	208	208
query54	644	638	551	551
query55	102	93	96	93
query56	367	331	338	331
query57	1206	1221	1122	1122
query58	293	285	289	285
query59	2607	2726	2585	2585
query60	379	367	363	363
query61	173	167	181	167
query62	801	720	647	647
query63	242	201	202	201
query64	4427	1292	1082	1082
query65	4336	4230	4241	4230
query66	1107	457	358	358
query67	15562	15199	15321	15199
query68	9530	961	600	600
query69	508	348	297	297
query70	1252	1185	1131	1131
query71	588	354	333	333
query72	5620	5006	5195	5006
query73	772	694	368	368
query74	9112	8903	8727	8727
query75	4455	3177	2668	2668
query76	3714	1166	750	750
query77	963	431	342	342
query78	9698	9677	8837	8837
query79	2447	854	624	624
query80	739	597	535	535
query81	507	278	235	235
query82	464	148	113	113
query83	323	267	300	267
query84	310	112	91	91
query85	907	520	447	447
query86	395	319	318	318
query87	4296	4232	4155	4155
query88	3529	2254	2260	2254
query89	410	347	309	309
query90	1833	232	240	232
query91	164	168	145	145
query92	101	78	78	78
query93	1954	1021	640	640
query94	708	458	339	339
query95	415	347	348	347
query96	492	608	295	295
query97	2663	2726	2573	2573
query98	271	222	223	222
query99	1439	1419	1293	1293
Total cold run time: 281439 ms
Total hot run time: 187789 ms

@doris-robot
Copy link

doris-robot commented Sep 4, 2025

ClickBench: Total hot run time: 32.38 s 
machine: 'aliyun_ecs.c7a.8xlarge_32C64G'
scripts: https://github.com/apache/doris/tree/master/tools/clickbench-tools
ClickBench test result on commit 9a09e9bb45b9dd4df851767414643c079614e0f7, data reload: false

query1	0.06	0.05	0.04
query2	0.10	0.06	0.06
query3	0.25	0.08	0.08
query4	1.61	0.12	0.13
query5	0.44	0.42	0.43
query6	1.18	0.65	0.66
query7	0.04	0.02	0.02
query8	0.06	0.05	0.05
query9	0.64	0.52	0.52
query10	0.58	0.57	0.60
query11	0.17	0.12	0.12
query12	0.15	0.12	0.12
query13	0.63	0.63	0.62
query14	0.82	0.88	0.83
query15	0.88	0.87	0.85
query16	0.41	0.40	0.38
query17	1.03	1.08	1.08
query18	0.22	0.21	0.20
query19	1.92	1.83	1.81
query20	0.01	0.01	0.02
query21	15.44	0.94	0.59
query22	0.80	1.13	0.74
query23	14.94	1.39	0.63
query24	6.88	2.29	0.40
query25	0.32	0.27	0.06
query26	0.54	0.16	0.13
query27	0.06	0.05	0.06
query28	9.30	0.94	0.43
query29	12.78	3.90	3.20
query30	3.05	3.07	2.99
query31	2.84	0.61	0.38
query32	3.24	0.57	0.47
query33	3.07	3.11	3.19
query34	16.02	5.48	4.83
query35	4.89	4.95	4.96
query36	0.71	0.53	0.50
query37	0.11	0.07	0.07
query38	0.06	0.04	0.04
query39	0.04	0.03	0.03
query40	0.18	0.15	0.14
query41	0.09	0.03	0.03
query42	0.03	0.03	0.03
query43	0.04	0.04	0.03
Total cold run time: 106.63 s
Total hot run time: 32.38 s

@shuke987
Copy link
Collaborator Author

shuke987 commented Sep 5, 2025

run p0

@shuke987
Copy link
Collaborator Author

shuke987 commented Sep 5, 2025

#55634

@shuke987 shuke987 closed this Sep 5, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@zclllyybb zclllyybb Awaiting requested review from zclllyybb zclllyybb will be requested when the pull request is marked ready for review zclllyybb is a code owner

@BiteTheDDDDt BiteTheDDDDt Awaiting requested review from BiteTheDDDDt BiteTheDDDDt will be requested when the pull request is marked ready for review BiteTheDDDDt is a code owner

@gavinchou gavinchou Awaiting requested review from gavinchou gavinchou will be requested when the pull request is marked ready for review gavinchou is a code owner

@dataroaring dataroaring Awaiting requested review from dataroaring dataroaring will be requested when the pull request is marked ready for review dataroaring is a code owner

@w41ter w41ter Awaiting requested review from w41ter w41ter will be requested when the pull request is marked ready for review w41ter is a code owner

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@shuke987 @Thearas @doris-robot